Codecov Report

Merging #5201 (5c43e28) into main (eaf5853) will increase coverage by 11.01%.
The diff coverage is 0.00%.

❗ Current head 5c43e28 differs from pull request most recent head e2767bb. Consider uploading reports for the commit e2767bb to get more accurate results

@@             Coverage Diff             @@
##             main    #5201       +/-   ##
===========================================
+ Coverage   72.62%   83.64%   +11.01%     
===========================================
  Files         274      274               
  Lines       39583    39591        +8     
===========================================
+ Hits        28749    33116     +4367     
+ Misses      10834     6475     -4359     

Flags with carried forward coverage won't be shown. Click here to find out more.

... and 73 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@philippjfr, when we are in the part of the codebase, I was thinking if we should change the colab detection from:

try:
    import google.colab  # noqa
    config.comms = "colab"
    return
except ImportError:
    pass

To:

if "google.colab" in sys.modules:
    config.comms = "colab"
    return

Timing in a normal notebook:

Check in a colab notebook:

Yep makes sense, didn't know it was automatically imported.